Skip to content

igvm_defs: add IGVM_VHT_OPTIONAL_BIT const#121

Merged
chris-oo merged 2 commits into
microsoft:mainfrom
luigix25:igvm_optional
Jun 8, 2026
Merged

igvm_defs: add IGVM_VHT_OPTIONAL_BIT const#121
chris-oo merged 2 commits into
microsoft:mainfrom
luigix25:igvm_optional

Conversation

@luigix25

@luigix25 luigix25 commented May 25, 2026

Copy link
Copy Markdown
Contributor

Bit 31 of a Variable Header Type indicates the header is optional and can be ignored by loaders that do not recognize it.

Add it as a const. This will be automatically exposed as a convenience macro in the generated C header.

@luigix25 luigix25 requested a review from a team as a code owner May 25, 2026 12:47
@luigix25

Copy link
Copy Markdown
Contributor Author

This is part of an ongoing to support optional directives in QEMU (https://lore.kernel.org/qemu-devel/20260525-igvm_optional-v1-1-28edc607acfd@redhat.com)

Bit 31 of a Variable Header Type indicates the header is optional
and can be ignored by loaders that do not recognize it.

Add it as a const. This will be automatically exposed as a
convenience macro in the generated C header.

Signed-off-by: Luigi Leonardi <leonardi@redhat.com>
@luigix25 luigix25 changed the title igvm_c: add IGVM_VHT_OPTIONAL_BIT macro to C igvm_c: add IGVM_VHT_OPTIONAL_BIT const May 25, 2026
@luigix25

luigix25 commented Jun 8, 2026

Copy link
Copy Markdown
Contributor Author

@chris-oo could you take a look? thanks!

@luigix25 luigix25 changed the title igvm_c: add IGVM_VHT_OPTIONAL_BIT const igvm: add IGVM_VHT_OPTIONAL_BIT const Jun 8, 2026
@luigix25 luigix25 changed the title igvm: add IGVM_VHT_OPTIONAL_BIT const igvm_defs: add IGVM_VHT_OPTIONAL_BIT const Jun 8, 2026
@chris-oo

chris-oo commented Jun 8, 2026

Copy link
Copy Markdown
Member

Sorry about the delay. I signed off on it, will merge.

@luigix25

luigix25 commented Jun 8, 2026

Copy link
Copy Markdown
Contributor Author

thanks!

@chris-oo chris-oo enabled auto-merge (squash) June 8, 2026 17:18
@chris-oo chris-oo merged commit f89a890 into microsoft:main Jun 8, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants